Skip to content

Conversation

@WojciechMat
Copy link
Collaborator

@WojciechMat WojciechMat commented Jun 16, 2025

Delete single filter button functionality

Implements working delete ("x") buttons for individual filter chips on the Events Discovery page.

Changes:

  • Added onDeleted callbacks to Location, Date Range, and Price filter chips
  • Replaced copyWith() calls with explicit EventFilterModel constructors to ensure proper filter removal
  • Each delete button now properly clears its specific filter while preserving other active filters
  • Maintains pagination state reset and triggers event reload after filter removal

Fixes: Filter chips now have functional delete buttons that actually remove the applied filters instead of being non-interactive.

@WojciechMat WojciechMat changed the base branch from main to dev June 16, 2025 12:35
@kryczkal kryczkal requested a review from Copilot June 16, 2025 12:52
Copy link
Collaborator

@kryczkal kryczkal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Adds fully functional delete buttons to the filter chips on the Events Discovery page, replacing previous non-interactive UI elements.

  • Implements onDeleted callbacks for Location, Date Range, and Price chips
  • Replaces copyWith() with explicit EventFilterModel constructors to clear individual filters
  • Resets pagination and reloads events after filter removal

@WojciechMat WojciechMat merged commit 1f135c6 into dev Jun 16, 2025
2 checks passed
@WojciechMat WojciechMat deleted the WojciechMat/delete-single-filter-button branch June 16, 2025 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants